Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix #732, change uint32 to size_t #733

Merged
merged 1 commit into from
Jan 13, 2021
Merged

Fix #732, change uint32 to size_t #733

merged 1 commit into from
Jan 13, 2021

Conversation

zanzaben
Copy link
Contributor

@zanzaben zanzaben commented Jan 4, 2021

Describe the contribution
Fixes #732

change prototypes should be the same, from uint32 to size_t

Testing performed
Build on RTEMS 5

Expected behavior changes
Able to build

System(s) tested on
Ubuntu 20.04 as build host for RTEMS 5.0.0

Additional context
Dependant on #704 to successfully build

Contributor Info - All information REQUIRED for consideration of pull request
Alex Campbell GSFC

@astrogeco
Copy link
Contributor

@zanzaben is this ready for discussion?

@zanzaben zanzaben added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Jan 6, 2021
@astrogeco astrogeco added CCB-20210106 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Jan 6, 2021
@astrogeco
Copy link
Contributor

CCB 2021-01-06 APPROVED

@zanzaben zanzaben force-pushed the Fix732_rtems_5_conflicting_types branch from 2b4fc63 to efd7f35 Compare January 11, 2021 18:54
@zanzaben zanzaben changed the title Fix #732 change uint32 to size_t Fix #732, change uint32 to size_t Jan 12, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate January 13, 2021 02:47
@astrogeco astrogeco merged commit b1f1b27 into nasa:integration-candidate Jan 13, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Jan 13, 2021
@astrogeco
Add nasa/osal#733
c0a147b
@astrogeco astrogeco mentioned this pull request Jan 13, 2021
Merged
@zanzaben zanzaben deleted the Fix732_rtems_5_conflicting_types branch January 14, 2021 13:31
@skliper skliper added this to the 6.0.0 milestone Sep 24, 2021
jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
@skliper
Fix nasa#711, Update SB/CCSDS FSW for msg module
8563679 
- Fix nasa#733: Validate checksum description update
- Fix nasa#597: Remove local endian SID macros
- Updates SB to use msg module
- General cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dependency
Projects
None yet
Milestone
Caelum
Development

Successfully merging this pull request may close these issues.

RTEMS 5 needs size_t type to build
3 participants
@zanzaben @astrogeco @skliper